Skip to content

Conversation

@shanshin
Copy link
Contributor

No description provided.

@shanshin shanshin force-pushed the gradle-9 branch 2 times, most recently from 04b0737 to 1d3689f Compare September 18, 2025 14:13
@shanshin
Copy link
Contributor Author

New limitation: minimal JVM version is 17

})
// TODO fix the migration issue
// source(compileTask.map {
// @Suppress("INVISIBLE_MEMBER")
Copy link
Contributor Author

@shanshin shanshin Sep 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Tapchicoma, could you please help with fixing this problem?

it.commonSourceSet
})
// TODO fix the migration issue
// @Suppress("INVISIBLE_MEMBER")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

val generator: TaskProvider<Task?> = null
setModuleDescriptorGenerator(generator)
// TODO fix the migration issue
// @Suppress("NULL_FOR_NONNULL_TYPE")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And here

@sandwwraith
Copy link
Member

@Tapchicoma PTAL

@@ -1,11 +1,11 @@
[versions]
kotlin = "2.2.0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest first updating this repo to Kotlin 2.2.21, as it contains an important fix for Gradle 9.0.0 support

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok.
However, the problems with Java modularity are not due to Gradle compatibility - they are due to using the outdated KGP API and I'm not aware how to correct them correctly.

And one problem is that @Suppress("NULL_FOR_NONNULL_TYPE") does not work in K2 - perhaps there are also ready-made solutions here on how to do this in a different way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants